Skip to content

chore(preprod): Improve error messaging for zip normalizing step in case of duplicate filename case#2864

Merged
NicoHinderling merged 2 commits intomasterfrom
improve-error-messaging-for-normalizing-step
Oct 23, 2025
Merged

chore(preprod): Improve error messaging for zip normalizing step in case of duplicate filename case#2864
NicoHinderling merged 2 commits intomasterfrom
improve-error-messaging-for-normalizing-step

Conversation

@NicoHinderling
Copy link
Copy Markdown
Contributor

If a user hits this case, they will have more context for what's going wrong

@NicoHinderling NicoHinderling requested review from a team as code owners October 22, 2025 21:28
Comment thread src/utils/build/normalize.rs Outdated
Changed `&zip_path` to `zip_path.as_str()` when calling `add_symlink`
and `start_file` methods. Passing `&String` instead of `&str` is not
idiomatic and could cause trait bound issues with `Into<String>`.

Also fixed clippy warnings for uninlined format args.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@runningcode runningcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@NicoHinderling NicoHinderling merged commit e6428dc into master Oct 23, 2025
27 checks passed
@NicoHinderling NicoHinderling deleted the improve-error-messaging-for-normalizing-step branch October 23, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants